Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving hackernews examples to stable #2487

Closed
wants to merge 4 commits into from
Closed

Moving hackernews examples to stable #2487

wants to merge 4 commits into from

Conversation

Alt-iOS
Copy link

@Alt-iOS Alt-iOS commented Apr 2, 2024

No description provided.

@Alt-iOS
Copy link
Author

Alt-iOS commented Apr 3, 2024

Hey so with the stable specification in the rust-toolchain.toml it was getting an error that said target wasm32-unknown-unknown not installed so maybe it was searching for the latest stable version and thats not with what the CI is config to use, thats why I took that out in the latest commit, hoping now it does pass

@benwis
Copy link
Contributor

benwis commented Apr 5, 2024

I wouldn't worry about the CI, change it to what you'd expect and we'll fix the CI. Looks like you have some conflicts too

@gbj
Copy link
Collaborator

gbj commented Apr 5, 2024

Thanks for the contribution! It looks like it has crossed paths with other contributions, so I'm going to close it now -- sorry about that.

@gbj gbj closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants